Closed
Conversation
Trezor switched to uv for dependency management
Recent versions of Speculos no longer support the Nano S. Additionally, we build the legacy app for the Nano S separately now.
lwip is a submodule of many device firmware repos, but the default URLs often fail to be cloned. Rewrite those URLs with git config to use the Github mirror to avoid that issue.
Instead of listing out the dependencies required to install manually, just install from speculos itself.
These don't seem to work anymore.
|
How about we tag / bring in the maintainers for the projects for which the CI is failing, it's mainly Coldcard apparently. |
Merged
Member
Member
|
Managed to get #818 green. |
achow101
added a commit
that referenced
this pull request
Jan 29, 2026
cafc8b0 ci, bitbox02: stop simulator after timeout (Sjors Provoost) 38fae57 Constrain cbor2 to <5.8 (Sjors Provoost) 51e55a1 test, bitbox02: add 5 minute startup timeout (Sjors Provoost) 85d98b2 test, jade: add 5 minute startup timeout (Sjors Provoost) 27f59da ci, jade: pin to v1.0.36 (Sjors Provoost) ba35c93 ci, coldcard: pin to v5.4.4 (Sjors Provoost) 511e019 ci, ledger: Pin Ledger app versions for compatibility (Sjors Provoost) cab1a9a ci: Pin device firmware versions in setup_environment.sh (Sjors Provoost) d25cc7d test: assert getkeypool returns list before passing to importdescriptors (Sjors Provoost) 3d140ec test, coldcard: Disable external input tests (Ava Chow) 613fe84 ci: install some more dependencies (Ava Chow) 2491a38 test, coldcard: Apply mpy patch before building (Ava Chow) 9ba9a17 test, ledger: Update automation (Ava Chow) 1d09f40 test, ledger: Install speculos dependencies from speculos (Ava Chow) 5331398 test, keepkey: Patch nanopb deprecated file mode (Ava Chow) c6c5559 ci, test: Use shallow clones for device firmware repos (Ava Chow) d4f1873 ci: Use github mirror of lwip (Ava Chow) 54b8869 ci, ledger: Build Bitcoin Legacy app and use nanosp for testing it (Ava Chow) 18b747e ci: Fix dist tests (Ava Chow) 8e63078 ci: Stop using ubuntu-22.04 (Ava Chow) 09080f2 test, trezor: Patch Trezor T to build (Ava Chow) 11ade3a test, trezor: Change to uv (Ava Chow) c070643 test, coldcard: Update multisig patch (Ava Chow) Pull request description: Time to stop the whack-a-mole game. Pin firmware and simulator versions to the latest releases known to work. We or the device makers can bump them as needed. This approach seems more suitable for the lower pace of development on this repo. Superseedes #812 (takes most of its commits) ACKs for top commit: achow101: ACK cafc8b0 Tree-SHA512: b6646cddc2a6c8fd68956aba1d35343fc4ef9531aee4b4fdc7668c74d559467cd113121b5b17f9cff6519f2491da7fe32057534e2a9dccd901257ec6c715f5bb
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
A number of things changed recently, breaking CI again.
Coldcard is still failing some signing tests, looking into those.